Skip to content

feat(passkey-crypto): add removePasskeyFromAccount function#8639

Merged
derranW26 merged 1 commit intomasterfrom
passkey/ticket-6-remove-passkey-from-account
May 6, 2026
Merged

feat(passkey-crypto): add removePasskeyFromAccount function#8639
derranW26 merged 1 commit intomasterfrom
passkey/ticket-6-remove-passkey-from-account

Conversation

@derranW26
Copy link
Copy Markdown
Contributor

Summary

  • Adds modules/sdk-core/src/bitgo/passkey/ module with removePasskeyFromAccount, WebAuthnOtpDevice type, and barrel index.ts
  • removePasskeyFromAccount issues DELETE /api/v2/user/otp/{device.id} using device.id (not credentialId)
  • Exports passkey module from modules/sdk-core/src/bitgo/index.ts
  • Unit tests verify correct device.id usage and no credentialId leakage

Notes

  • WebAuthnOtpDevice is a local stub — swap to @bitgo/public-types once that PR merges (TODO comment in types.ts)
  • Callers must invoke removePasskeyFromWallet() for all affected wallets before calling this function (JSDoc warning on the function)

Ticket

WCN-191

@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from a2b2564 to c18a4d4 Compare April 28, 2026 14:39
@derranW26 derranW26 changed the base branch from passkey/ticket-1-passkey-crypto-package to master April 28, 2026 14:39
@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from c18a4d4 to 98dfc2d Compare May 4, 2026 05:00
@derranW26 derranW26 marked this pull request as ready for review May 4, 2026 05:06
@derranW26 derranW26 requested review from a team as code owners May 4, 2026 05:06
@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from 23382f8 to 2f80518 Compare May 4, 2026 14:25
@derranW26
Copy link
Copy Markdown
Contributor Author

@claude review this pr

@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from 2f80518 to 7e7382f Compare May 4, 2026 18:09
- DELETE /user/otp/{device.id} to remove a passkey from the account
- Validates device.id before making the API call
- Imports BitGoBase from @bitgo/sdk-core

WCN-191
@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from 7e7382f to b062c83 Compare May 4, 2026 19:05
@derranW26 derranW26 changed the title feat(sdk-core): add removePasskeyFromAccount function feat(passkey-crypto): add removePasskeyFromAccount function May 5, 2026
@derranW26 derranW26 merged commit a613d8b into master May 6, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants